Skip to content

chore(toolbox-core): Remove unused typevar variables #232

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
May 16, 2025

Conversation

anubhav756
Copy link
Contributor

No description provided.

@anubhav756 anubhav756 self-assigned this May 8, 2025
@anubhav756 anubhav756 requested a review from a team as a code owner May 8, 2025 12:15
@anubhav756 anubhav756 changed the title chore: Remove unused typevar variables chore(toolbox-core): Remove unused typevar variables May 9, 2025
@anubhav756 anubhav756 force-pushed the anubhav-sync-add-header branch from 7bb2e88 to 5eae73f Compare May 12, 2025 14:13
@anubhav756 anubhav756 force-pushed the anubhav-sync-add-header branch from 5eae73f to 75321df Compare May 13, 2025 11:33
@anubhav756 anubhav756 force-pushed the anubhav-sync-add-header branch from 75321df to ab6900a Compare May 14, 2025 12:42
@anubhav756 anubhav756 force-pushed the anubhav-sync-add-header branch from ab6900a to d554a48 Compare May 16, 2025 15:49
@anubhav756 anubhav756 force-pushed the anubhav-sync-add-header branch from d554a48 to a6a24b8 Compare May 16, 2025 19:14
Base automatically changed from anubhav-sync-add-header to main May 16, 2025 19:17
This change introduces a warning that is displayed immediately before a tool invocation if:
1. The invocation includes an authentication header.
2. The connection is being made over non-secure HTTP.

> [!IMPORTANT]
The purpose of this warning is to alert the user to the security risk of sending credentials over an unencrypted channel and to encourage the use of HTTPS.
@anubhav756 anubhav756 merged commit aa69aab into main May 16, 2025
19 checks passed
@anubhav756 anubhav756 deleted the anubhav-typevar branch May 16, 2025 19:45
@release-please release-please bot mentioned this pull request May 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants